Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New test script and fixing of MacOSX problems #4

Merged
merged 48 commits into from
May 14, 2016

Conversation

greatfireball
Copy link
Member

@greatfireball greatfireball commented May 11, 2016

I create a new test.sh which behaves now like a TAP test. Therefore, perl's prove might be used to run all tests.

Moreover, I fixed the problems with readlink -f under MacOSX

Fixes #1
Fixes #2

@review-ninja
Copy link

ReviewNinja

@greatfireball
Copy link
Member Author

greatfireball commented May 11, 2016

Approved

Approved with PullApprove

@greatfireball greatfireball added this to the Version 0.4 milestone May 12, 2016
@greatfireball greatfireball force-pushed the feature/new-test-script branch 2 times, most recently from 9d27f76 to 6ead3e1 Compare May 12, 2016 08:31
Travis-CI test should run from now on
I am using the new feature to test under different operation systems.

Therefore, we should be able to guarantee that the SeqChunker works under Linux and MacOSX
According to https://docs.travis-ci.com/user/customizing-the-build/#Customizing-the-Installation-Step the installation step to resolve dependencies can be avoided. In our project that makes sense due to we have no real perl program running
greatfireball and others added 20 commits May 12, 2016 17:04
…nd moved input file definition down to tests
Changed the way, the combined output will be generated. So each test will
generate its own set of output files. The name of the output files are generated
according to:

	  tmp.TESTNUMBER.*

So the tests if the correct output was generated can now avoid diff, but use the
md5sum of the output instead.
greatfireball and others added 8 commits May 14, 2016 01:15
This is expected to fail for the original test script, but should work on the new one
To generate the sums based on the original one, I used the following perl statement:

```
perl -ne '

     unless (/tmp/) { print; next }

     print;

     foreach my $fact (map {$_*14} (1..3))
     {
	$bla=$_;
	$bla=~s/(?<=\.)(\d+)(?=\.)/$1+$fact/eg; print $bla;
     }

' reference.md5
```
@greatfireball greatfireball merged commit 2bdad9e into develop May 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants